Skip to content

feature/FOUR-9390: Restoring a User is not registered#5026

Merged
ryancooley merged 3 commits into
developfrom
feature/FOUR-9390
Jul 20, 2023
Merged

feature/FOUR-9390: Restoring a User is not registered#5026
ryancooley merged 3 commits into
developfrom
feature/FOUR-9390

Conversation

@pmPaulis
Copy link
Copy Markdown
Contributor

@pmPaulis pmPaulis commented Jul 19, 2023

Issue & Reproduction Steps

  • Delete a User
  • Restore User
  • Go to User Activity Logging

Solution

  • The restore action was registered in the security log

How to Test

  • Delete a User
  • Restore User
  • Go to User Activity Logging

Related Tickets & Packages

Code Review Checklist

  • I have pulled this code locally and tested it on my instance, along with any associated packages.
  • This code adheres to ProcessMaker Coding Guidelines.
  • This code includes a unit test or an E2E test that tests its functionality, or is covered by an existing test.
  • This solution fixes the bug reported in the original ticket.
  • This solution does not alter the expected output of a component in a way that would break existing Processes.
  • This solution does not implement any breaking changes that would invalidate documentation or cause existing Processes to fail.
  • This solution has been tested with enterprise packages that rely on its functionality and does not introduce bugs in those packages.
  • This code does not duplicate functionality that already exists in the framework or in ProcessMaker.
  • This ticket conforms to the PRD associated with this part of ProcessMaker.

@pmPaulis pmPaulis force-pushed the feature/FOUR-9390 branch from f206798 to 9bb0883 Compare July 19, 2023 20:34
@pmPaulis pmPaulis changed the title FOUR-9390: Restore user needs to log FOUR-9390: Restoring a User is not registered Jul 19, 2023
@pmPaulis pmPaulis force-pushed the feature/FOUR-9390 branch from 9bb0883 to b2e5668 Compare July 19, 2023 22:42
@pmPaulis pmPaulis changed the title FOUR-9390: Restoring a User is not registered feature/FOUR-9390: Restoring a User is not registered Jul 19, 2023
@pmPaulis pmPaulis requested a review from fagubla July 20, 2023 14:21
@pmPaulis pmPaulis force-pushed the feature/FOUR-9390 branch from 229cd68 to 58381d9 Compare July 20, 2023 15:35
@pmPaulis pmPaulis requested a review from CarliPinell July 20, 2023 15:35
@processmaker-sonarqube
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@CarliPinell CarliPinell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's ok for me.

@ryancooley ryancooley merged commit 0b09671 into develop Jul 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants